Skip to content

Conversation

@mdbirnstiehl
Copy link
Contributor

Partition suggestions require an LLM which was not previously noted.

@github-actions
Copy link

github-actions bot commented Nov 5, 2025

@mdbirnstiehl mdbirnstiehl marked this pull request as ready for review November 5, 2025 17:02
@mdbirnstiehl mdbirnstiehl requested a review from a team as a code owner November 5, 2025 17:02
Copy link
Contributor

@florent-leborgne florent-leborgne left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we check what from the security page can be reused here in observability? it doesn't feel right to link to a security page that starts with "Elastic Security", even if the content inside is relevant.

By using snippets, or by moving some of that content to a more central place?

@mdbirnstiehl
Copy link
Contributor Author

Can we check what from the security page can be reused here in observability? it doesn't feel right to link to a security page that starts with "Elastic Security", even if the content inside is relevant.

By using snippets, or by moving some of that content to a more central place?

@florent-leborgne How about if I just link to the new GenAI connectors docs for now. I think we reuse most of this info and create a centralized place for both AI assistants.

Copy link
Contributor

@florent-leborgne florent-leborgne left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@mdbirnstiehl mdbirnstiehl merged commit 54643a5 into elastic:main Nov 6, 2025
7 checks passed
@mdbirnstiehl mdbirnstiehl deleted the llm-note-partition branch November 6, 2025 15:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants